Skip to content

Conversation

@atlj
Copy link
Collaborator

@atlj atlj commented Nov 16, 2024

Summary

The previous index.ts file of create-react-native-library was too long and complicated to maintain. This PR splits the file a bit

Test plan

  1. We have to make sure that the behavior hasn't changed at all. CI pipeline is the best way to check this.

@atlj atlj changed the title refactor: crnl index file refactor(crnl): split the index file Nov 16, 2024
import fs from 'fs-extra';
import type { ExampleApp } from '../input';

export async function getDependencyVersionsFromExample(
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
export async function getDependencyVersionsFromExample(
export async function getDependencyVersionsFromExampleApp(

folder: string,
exampleAppType: ExampleApp
) {
// Set `react` and `react-native` versions of root `package.json` from example `package.json`
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Set `react` and `react-native` versions of root `package.json` from example `package.json`

@atlj atlj force-pushed the @atlj/refactor-crnl-main branch from ef29e22 to ffd2eed Compare November 29, 2024 09:12
@atlj atlj force-pushed the @atlj/refactor-crnl-main branch from ffd2eed to ffc4517 Compare November 29, 2024 09:15
@atlj atlj merged commit e969c89 into main Nov 29, 2024
21 of 29 checks passed
@atlj atlj deleted the @atlj/refactor-crnl-main branch November 29, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants